Skip to content

4.1 add routing context to hello message #701

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

gjmwoods
Copy link
Contributor

Include the routing context configured in the connection URI in the Hello message. This enables the server to make routing decisions with the same information provided to the driver.

For example with the URI neo4j://localhost:7687?policy=fast&region=eu the driver will send the map: {'policy': 'fast', 'region': 'eu'} in the Hello message's metadata with the key routing.

Since bolt is for direct connections the map is configured with {'routing': null}inform that the sever should not route queries.

@gjmwoods gjmwoods force-pushed the 4.1-add-routing-context-to-hello-message branch from 6a9fdf9 to b9697a1 Compare May 21, 2020 14:08
@gjmwoods gjmwoods force-pushed the 4.1-add-routing-context-to-hello-message branch from b9697a1 to 0a8b2e9 Compare May 26, 2020 12:20
@gjmwoods gjmwoods force-pushed the 4.1-add-routing-context-to-hello-message branch from 0a8b2e9 to 4504633 Compare May 28, 2020 08:11
Copy link
Contributor

@michael-simons michael-simons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍

@gjmwoods gjmwoods merged commit 034cb07 into neo4j:4.1 Jun 2, 2020
@gjmwoods gjmwoods deleted the 4.1-add-routing-context-to-hello-message branch June 2, 2020 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants